Allow apps_vars_file to be specified per application in config #2616
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hello,
[PR submitted in reference to Issue #2615]
While converting an old app from rebar to rebar3, I found some unexpected behavior from the app_vars_file configuration option in rebar3.
With this change, when apps_vars_file is a proplist, rebar3 will inspect the proplist for each app's name and only apply env vars if found. For backward compatibility, if app_vars_file is not a proplist, the vars will still be applied to all apps.
I've not attempted to create any new tests for this yet, but if the PR is deemed valuable I'm happy to include tests and documentation as needed.